Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert "Support editables in uv sync (#3692)" #3696

Merged
merged 1 commit into from
May 21, 2024
Merged

Revert "Support editables in uv sync (#3692)" #3696

merged 1 commit into from
May 21, 2024

Conversation

konstin
Copy link
Member

@konstin konstin commented May 21, 2024

This reverts commit #3692 until we get the editables from the lockfile.

@konstin konstin requested a review from charliermarsh May 21, 2024 13:37
@konstin konstin mentioned this pull request May 21, 2024
Merged
@charliermarsh charliermarsh enabled auto-merge (squash) May 21, 2024 13:52
@charliermarsh charliermarsh merged commit f396c6c into main May 21, 2024
44 checks passed
@charliermarsh charliermarsh deleted the konsti/revert-uv-sync-editable branch May 21, 2024 14:11
charliermarsh pushed a commit that referenced this pull request May 21, 2024
@konstin @charliermarsh
Revert "Support editables in uv sync (#3692)" (#3696)
7eb050e 
This reverts commit #3692 until we get the editables from the lockfile.
@charliermarsh charliermarsh mentioned this pull request May 21, 2024
Merged
charliermarsh added a commit that referenced this pull request May 21, 2024
@charliermarsh @konstin
Revert "Support editables in uv sync (#3692)" (#3696) (#3722)
e0b6398 
## Summary

This is just a re-apply of #3696, which @konstin accidentally reverted
in #3585.

Co-authored-by: konsti <konstin@mailbox.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charliermarsh charliermarsh charliermarsh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@konstin @charliermarsh